Skip to content

Fix open security alerts - #463

Merged
hansolosan merged 1 commit into
masterfrom
fix/security-alerts
Nov 13, 2026
Merged

hansolosan merged 1 commit into
masterfrom
fix/security-alerts

Conversation

@hansolosan

Copy link
Copy Markdown
Collaborator

Summary

Resolves all currently open Dependabot and code scanning alerts.

Dependabot (#8–#12)

Bump tools.jackson.core:jackson-databind and tools.jackson.dataformat:jackson-dataformat-xml from 3.2.1 to 3.2.3. Addresses:

  • Quadratic forward-reference completion (high)
  • Unbounded retention of unknown raw type IDs (high)
  • Duration / XMLGregorianCalendar unbounded number parse DoS (high)
  • Comparable missing from DefaultBaseTypeLimitingValidator unsafe base types (medium)
  • Path deserialization missing scheme allowlist (medium)

Code scanning

  • Update the README.md #2 actions/missing-workflow-permissions: add permissions: contents: read to .github/workflows/mvntest.yml.
  • Initial code check in for JSONata4Java #1 java/implicit-cast-in-compound-assignment: DateTimeUtils.lettersToDecimal used decimal += ... * Math.pow(26, i) (implicit double→int narrowing). Replaced with integer arithmetic using a running place value; output verified identical to the previous implementation on sample inputs.

Testing

mvn --batch-mode verify: BUILD SUCCESS, 1223 tests run, 0 failures, 0 errors (93 skipped in AgnosticTestSuite).

- Bump jackson-databind and jackson-dataformat-xml from 3.2.1 to 3.2.3
  to resolve Dependabot alerts #8-#12 (DoS issues, incomplete
  PolymorphicTypeValidator denylist, Path deserialization scheme checks).
- Restrict the CI workflow token to `contents: read`
  (code scanning alert actions/missing-workflow-permissions).
- Replace implicit double-to-int compound assignment in
  DateTimeUtils.lettersToDecimal with integer arithmetic
  (code scanning alert java/implicit-cast-in-compound-assignment).

Signed-off-by: Radu Florian <raduf@us.ibm.com>
@hansolosan
hansolosan merged commit 8accd6f into master Nov 13, 2026
6 checks passed
@hansolosan
hansolosan deleted the fix/security-alerts branch November 13, 2026 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant